Skip to content

Conversation

@samsonasik
Copy link
Member

@samsonasik samsonasik commented Dec 22, 2025

Current composer.json already require phpstan 2.1.34, so this tweak AlwaysRememberedExpr no longer needed.

Per PHPStan PR:

@samsonasik
Copy link
Member Author

Tested as well on rector- packages:

  • rector-phpunit
  • rector-downgrade-php

with require-dev:

"phpstan/phpstan": "2.1.x-dev",
"rector/rector-src": "dev-test-phpstan-21x",

and seems working ok 👍

Screenshot 2025-12-22 at 21 38 45 Screenshot 2025-12-22 at 21 38 51

@TomasVotruba TomasVotruba marked this pull request as draft January 13, 2026 14:09
@TomasVotruba
Copy link
Member

Note: waits on next PHPStan release, after 2.1.33

@samsonasik samsonasik changed the title Test PHPStan 2.1.x-dev to remove tweak AlwaysRememberedExpr on Printer Bump to PHPStan ^2.1.34 to remove tweak AlwaysRememberedExpr on Printer Jan 19, 2026
@samsonasik samsonasik marked this pull request as ready for review January 19, 2026 20:05
@samsonasik
Copy link
Member Author

@TomasVotruba PHPStan 2.1.34 just released phpstan/phpstan#13976

It seems there is error on PHPStanContainerMemento

https://github.com/rectorphp/rector-src/actions/runs/21150113451/job/60824192355?pr=7770

I will check on separate PR.

@samsonasik samsonasik marked this pull request as draft January 19, 2026 21:09
@samsonasik
Copy link
Member Author

wait for PR:

to be merged first for fix compatible code.

@samsonasik samsonasik marked this pull request as ready for review January 19, 2026 21:35
@samsonasik
Copy link
Member Author

I will rebase

@samsonasik samsonasik changed the title Bump to PHPStan ^2.1.34 to remove tweak AlwaysRememberedExpr on Printer Remove no longer tweak AlwaysRememberedExpr on Printer per PHPStan 2.1.34 Jan 19, 2026
@samsonasik
Copy link
Member Author

Finally, All green 🎉

@TomasVotruba ready 👍

@samsonasik samsonasik changed the title Remove no longer tweak AlwaysRememberedExpr on Printer per PHPStan 2.1.34 Remove no longer needed tweak AlwaysRememberedExpr on Printer per PHPStan 2.1.34 Jan 19, 2026
@TomasVotruba TomasVotruba merged commit 38d1caf into main Jan 19, 2026
57 checks passed
@TomasVotruba
Copy link
Member

Thank you 👍

@TomasVotruba TomasVotruba deleted the test-phpstan-21x branch January 19, 2026 22:17
calebdw added a commit to calebdw/rector-src that referenced this pull request Jan 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants